Skip to content

docs(readme): call out C toolchain prerequisite for source builds#92

Merged
aksOps merged 1 commit intomainfrom
docs/cgo-prereq-readme
May 3, 2026
Merged

docs(readme): call out C toolchain prerequisite for source builds#92
aksOps merged 1 commit intomainfrom
docs/cgo-prereq-readme

Conversation

@aksOps
Copy link
Copy Markdown
Contributor

@aksOps aksOps commented May 3, 2026

Summary

  • Add a Prerequisites paragraph to the README "Build from source" section.
  • Names the misleading symptom (undefined: sqlitevec.LoadInto) so anyone hitting it via search lands on the fix.
  • Points at the existing comprehensive prerequisites list in docs/getting-started.md.

Why

internal/sqlitevec/load.go is gated by //go:build cgo. Without gcc on PATH, CGO is silently disabled and the build fails at the call site (cmd/serve.go:102) with undefined: sqlitevec.LoadInto instead of a clear toolchain error. Hit this in practice today on a fresh box.

Test plan

  • git diff is README-only, additive
  • CI green
  • Reviewer eyeballs the wording

Without gcc/clang on PATH, CGO is silently disabled and `go build` /
`go install` fails at the call site with `undefined: sqlitevec.LoadInto`
rather than a clear toolchain error, because `internal/sqlitevec/load.go`
is gated by `//go:build cgo`. The README's "Build from source" section
already assumed CGO_ENABLED=1 in the command but never said how to make
that actually work — leaving the next from-source builder to chase a
misleading "undefined symbol" error.

Add a short prerequisites paragraph above the build commands, pointing
at the existing comprehensive list in docs/getting-started.md.
@aksOps aksOps enabled auto-merge (squash) May 3, 2026 16:03
@aksOps aksOps merged commit 3c6b046 into main May 3, 2026
16 of 17 checks passed
@aksOps aksOps deleted the docs/cgo-prereq-readme branch May 3, 2026 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant